The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jul 22, 2026 10:21pm

PR Summary

Low Risk
Test-only refactor with no application code changes; risk is limited to test fidelity if a migrated mock no longer matches real query ordering.

Overview
Tranche 3 of the shared database test mock rollout: 27 test suites in apps/sim stop defining their own Drizzle-style vi.mock('@sim/db', …) chains and instead use @sim/testing (dbChainMock, dbChainMockFns, queueTableRows, resetDbChainMock, and often schemaMock).

Coverage spans EE (permission checks, workspace forking cleanup/unlink), lib/workflows (deploy, lifecycle, duplicate, schedules, deployment outbox, execution state, skills), lib/copilot (chat, stream liveness, agent URL, knowledge/tools), plus auth/ban, admin, BYOK, outbox, idempotency cleanup, organizations, uploads, and workspace admin-move. Tests now queue rows by schema table and assert via dbChainMockFns (e.g. set, insert, transaction) instead of hand-rolled mockLimit / mockWhere / custom createMockTx helpers.

workspace-context.test.ts drops a no-op @sim/db mock entirely. afterAll(resetDbChainMock) and beforeEach(resetDbChainMock) are added consistently so queued rows and spy state do not leak across tests.

No production/runtime behavior changes—only test setup and assertions.

^{Reviewed by Cursor Bugbot for commit 1000474. Configure here.}

Greptile Summary

This PR moves 27 test suites onto the shared database mock utilities. The main changes are:

• Replaces suite-local @sim/db factories with dbChainMock.
• Queues select results by schema table with queueTableRows.
• Standardizes mock reset, transaction, mutation, and operator assertions.

Confidence Score: 5/5

This looks safe to merge.

• No blocking issues found in the changed code.
• The queued rows follow the current production query order.
• Reset and mutation behavior remain consistent with the prior test factories.

Important Files Changed

_{Reviews (1): Last reviewed commit: "improvement(tests): use the shared notLi..." | Re-trigger Greptile}